Skip to content

[CALCITE-6831] Add ARRAY_SLICE function(enabled in Hive Library)#4194

Merged
NobiGo merged 1 commit intoapache:mainfrom
xuzifu666:array_splice_support
Feb 17, 2025
Merged

[CALCITE-6831] Add ARRAY_SLICE function(enabled in Hive Library)#4194
NobiGo merged 1 commit intoapache:mainfrom
xuzifu666:array_splice_support

Conversation

@xuzifu666
Copy link
Copy Markdown
Member

Comment thread core/src/main/java/org/apache/calcite/runtime/SqlFunctions.java
Comment thread testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java
Comment thread core/src/main/java/org/apache/calcite/runtime/SqlFunctions.java Outdated
Comment thread testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java
Comment thread testkit/src/main/java/org/apache/calcite/test/SqlOperatorTest.java
Copy link
Copy Markdown
Member

@caicancai caicancai left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@ILuffZhe ILuffZhe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LTGM! We can merge this after squashing commits.

@caicancai caicancai added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Feb 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants